Skip to content

refactor(preview): remove deprecated PreviewManager::registerProvider API#59905

Open
joshtrichards wants to merge 3 commits intomasterfrom
jtr/chore-drop-registerProvider-IPreview
Open

refactor(preview): remove deprecated PreviewManager::registerProvider API#59905
joshtrichards wants to merge 3 commits intomasterfrom
jtr/chore-drop-registerProvider-IPreview

Conversation

@joshtrichards
Copy link
Copy Markdown
Member

@joshtrichards joshtrichards commented Apr 24, 2026 •
edited
Loading

🧹

Summary

  • remove the deprecated public PreviewManager::registerProvider() API
  • replace it with a private internal helper for provider registration
  • update internal core and bootstrap provider registration to use the new helper
  • add an explicit return type to the core provider closure for consistency with bootstrap provider registration

Notes:

  • keeps the existing provider registration behavior unchanged
  • reduces public API surface

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@joshtrichards joshtrichards added this to the Nextcloud 34 milestone Apr 24, 2026
@joshtrichards joshtrichards changed the title refactor(preview): remove deprecated PreviewManager::registerProvider API refactor(preview): remove deprecated PreviewManager::registerProvider API Apr 24, 2026
@joshtrichards
chore(previews): drop long deprecated registerProvider from IPreview
43427a4 
Signed-off-by: Josh <josh.t.richards@gmail.com>
@joshtrichards
chore(previews): registerProvider->registerProviderClosure in Preview…
86b471d 
…Manager

No longer part of the public interface. Just an internal utility function.

Apps/etc register in other ways and still end up here appropriately.

Signed-off-by: Josh <josh.t.richards@gmail.com>
@joshtrichards
chore(previews): let psalm know about $class typing
8e43019 
Signed-off-by: Josh <josh.t.richards@gmail.com>
@joshtrichards joshtrichards force-pushed the jtr/chore-drop-registerProvider-IPreview branch from e952d51 to 8e43019 Compare April 25, 2026 15:48
@joshtrichards joshtrichards added the 3. to review Waiting for reviews label Apr 25, 2026
@joshtrichards joshtrichards marked this pull request as ready for review April 25, 2026 15:48
@joshtrichards joshtrichards requested a review from a team as a code owner April 25, 2026 15:48
@joshtrichards joshtrichards requested review from CarlSchwan, leftybournes, provokateurin and salmart-dev and removed request for a team April 25, 2026 15:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@salmart-dev salmart-dev Awaiting requested review from salmart-dev salmart-dev is a code owner automatically assigned from nextcloud/server-backend

@leftybournes leftybournes Awaiting requested review from leftybournes leftybournes is a code owner automatically assigned from nextcloud/server-backend

@CarlSchwan CarlSchwan Awaiting requested review from CarlSchwan CarlSchwan is a code owner automatically assigned from nextcloud/server-backend

@provokateurin provokateurin Awaiting requested review from provokateurin provokateurin is a code owner automatically assigned from nextcloud/server-backend

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

3. to review Waiting for reviews feature: previews and thumbnails technical debt 🧱 🤔🚀

Projects

None yet

Milestone

Nextcloud 34

Development

Successfully merging this pull request may close these issues.

1 participant

@joshtrichards